Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JS and TS changes for VS Code #151

Merged
merged 3 commits into from
Sep 11, 2017
Merged

JS and TS changes for VS Code #151

merged 3 commits into from
Sep 11, 2017

Conversation

pjlamb12
Copy link
Contributor

This PR fixes some JavaScript and TypeScript visual changes in VS Code. See attached pictures for the before and after.
screenshot 3
screenshot 4

@pjlamb12
Copy link
Contributor Author

Relates to #144

@codeofsumit
Copy link

@pjlamb12 nice PR - how can I install your PR version into vscode until it gets merged?

@dawsbot
Copy link

dawsbot commented Jul 30, 2017

What an awesome fix @pjlamb12! I'd love to see this merged so that the color-scheme is usable in vscode. I've had the task #144 open for 6 months at this point, and this would fix it!

@wesbos
Copy link
Owner

wesbos commented Aug 3, 2017

Asking for a few people to test this then I'll merge

@alexkrolick
Copy link

screen shot 2017-08-03 at 6 24 58 pm

screen shot 2017-08-03 at 6 23 49 pm

screen shot 2017-08-03 at 6 22 43 pm

@adriangmweb
Copy link

Finally any news with the merge?

@ghost
Copy link

ghost commented Aug 11, 2017

Might need some README changes if this works for VSCode as well as Sublime, @pjlamb12.

@alexkrolick
Copy link

@evanread I used the VSCode Yeoman generator (yo code) to import the Textmate theme as an extension for the screenshots above

@ghost
Copy link

ghost commented Aug 22, 2017

@wesbos You happy with the MR?

@wesbos
Copy link
Owner

wesbos commented Sep 11, 2017

looks good - thanks! I will push to VS code as soon as I can remember how...

@wesbos wesbos merged commit c628eb7 into wesbos:master Sep 11, 2017
@alexkrolick
Copy link

Are all the brackets supposed to be italics?

@wesbos
Copy link
Owner

wesbos commented Sep 11, 2017

nope - I'm getting into VS code so I'll give it a lookski

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants